Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated mod_revproxy #3617

Merged
merged 2 commits into from
Mar 30, 2022
Merged

Remove deprecated mod_revproxy #3617

merged 2 commits into from
Mar 30, 2022

Conversation

chrzaszcz
Copy link
Member

This module is dead - it was impossible to configure it since version 4.2.0

@mongoose-im
Copy link
Collaborator

mongoose-im commented Mar 30, 2022

small_tests_24 / small_tests / e23b199
Reports root / small


small_tests_23 / small_tests / e23b199
Reports root / small


dynamic_domains_mysql_redis_24 / mysql_redis / e23b199
Reports root/ big
OK: 2833 / Failed: 0 / User-skipped: 150 / Auto-skipped: 0


dynamic_domains_pgsql_mnesia_23 / pgsql_mnesia / e23b199
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


dynamic_domains_pgsql_mnesia_24 / pgsql_mnesia / e23b199
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


dynamic_domains_mssql_mnesia_24 / odbc_mssql_mnesia / e23b199
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


ldap_mnesia_23 / ldap_mnesia / e23b199
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 400 / Auto-skipped: 0


ldap_mnesia_24 / ldap_mnesia / e23b199
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 400 / Auto-skipped: 0


internal_mnesia_24 / internal_mnesia / e23b199
Reports root/ big
OK: 1547 / Failed: 0 / User-skipped: 359 / Auto-skipped: 0


mysql_redis_24 / mysql_redis / e23b199
Reports root/ big
OK: 3219 / Failed: 0 / User-skipped: 147 / Auto-skipped: 0


pgsql_mnesia_23 / pgsql_mnesia / e23b199
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


elasticsearch_and_cassandra_24 / elasticsearch_and_cassandra_mnesia / e23b199
Reports root/ big
OK: 1854 / Failed: 0 / User-skipped: 367 / Auto-skipped: 0


pgsql_mnesia_24 / pgsql_mnesia / e23b199
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


mssql_mnesia_24 / odbc_mssql_mnesia / e23b199
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


riak_mnesia_24 / riak_mnesia / e23b199
Reports root/ big
OK: 1697 / Failed: 0 / User-skipped: 366 / Auto-skipped: 0

@codecov
Copy link

codecov bot commented Mar 30, 2022

Codecov Report

Merging #3617 (ab4b9fc) into master (66a1652) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3617      +/-   ##
==========================================
+ Coverage   80.94%   80.95%   +0.01%     
==========================================
  Files         425      424       -1     
  Lines       32219    32061     -158     
==========================================
- Hits        26079    25955     -124     
+ Misses       6140     6106      -34     
Impacted Files Coverage Δ
...stem_metrics/mongoose_system_metrics_collector.erl 93.81% <ø> (ø)
src/pubsub/mod_pubsub.erl 72.95% <0.00%> (-0.07%) ⬇️
src/inbox/mod_inbox_rdbms_async.erl 64.40% <0.00%> (ø)
src/mod_bosh_socket.erl 78.88% <0.00%> (+0.31%) ⬆️
src/mod_muc_rdbms.erl 95.93% <0.00%> (+0.81%) ⬆️
src/mod_muc.erl 75.34% <0.00%> (+0.92%) ⬆️
src/muc_light/mod_muc_light_db_mnesia.erl 90.52% <0.00%> (+1.05%) ⬆️
src/cassandra/mongoose_cassandra_worker.erl 68.54% <0.00%> (+2.81%) ⬆️
src/cassandra/mongoose_cassandra.erl 81.48% <0.00%> (+3.70%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66a1652...ab4b9fc. Read the comment docs.

@chrzaszcz chrzaszcz force-pushed the delete-mod-revproxy branch 2 times, most recently from 4cd19d2 to f7d56ec Compare March 30, 2022 11:36
@mongoose-im
Copy link
Collaborator

mongoose-im commented Mar 30, 2022

small_tests_24 / small_tests / 4cd19d2
Reports root / small


small_tests_23 / small_tests / 4cd19d2
Reports root / small


dynamic_domains_pgsql_mnesia_23 / pgsql_mnesia / 4cd19d2
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


dynamic_domains_mysql_redis_24 / mysql_redis / 4cd19d2
Reports root/ big
OK: 2833 / Failed: 0 / User-skipped: 150 / Auto-skipped: 0


dynamic_domains_pgsql_mnesia_24 / pgsql_mnesia / 4cd19d2
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


dynamic_domains_mssql_mnesia_24 / odbc_mssql_mnesia / 4cd19d2
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


ldap_mnesia_23 / ldap_mnesia / 4cd19d2
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 400 / Auto-skipped: 0


ldap_mnesia_24 / ldap_mnesia / 4cd19d2
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 400 / Auto-skipped: 0


internal_mnesia_24 / internal_mnesia / 4cd19d2
Reports root/ big
OK: 1547 / Failed: 0 / User-skipped: 359 / Auto-skipped: 0


pgsql_mnesia_23 / pgsql_mnesia / 4cd19d2
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


elasticsearch_and_cassandra_24 / elasticsearch_and_cassandra_mnesia / 4cd19d2
Reports root/ big
OK: 1854 / Failed: 0 / User-skipped: 367 / Auto-skipped: 0


pgsql_mnesia_24 / pgsql_mnesia / 4cd19d2
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


mysql_redis_24 / mysql_redis / 4cd19d2
Reports root/ big
OK: 3219 / Failed: 0 / User-skipped: 147 / Auto-skipped: 0


mssql_mnesia_24 / odbc_mssql_mnesia / 4cd19d2
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


riak_mnesia_24 / riak_mnesia / 4cd19d2
Reports root/ big
OK: 1697 / Failed: 0 / User-skipped: 366 / Auto-skipped: 0

@mongoose-im
Copy link
Collaborator

mongoose-im commented Mar 30, 2022

small_tests_24 / small_tests / f7d56ec
Reports root / small


small_tests_23 / small_tests / f7d56ec
Reports root / small


dynamic_domains_pgsql_mnesia_24 / pgsql_mnesia / f7d56ec
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


dynamic_domains_mysql_redis_24 / mysql_redis / f7d56ec
Reports root/ big
OK: 2833 / Failed: 0 / User-skipped: 150 / Auto-skipped: 0


dynamic_domains_pgsql_mnesia_23 / pgsql_mnesia / f7d56ec
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


dynamic_domains_mssql_mnesia_24 / odbc_mssql_mnesia / f7d56ec
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


internal_mnesia_24 / internal_mnesia / f7d56ec
Reports root/ big
OK: 1547 / Failed: 0 / User-skipped: 359 / Auto-skipped: 0


ldap_mnesia_24 / ldap_mnesia / f7d56ec
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 400 / Auto-skipped: 0


ldap_mnesia_23 / ldap_mnesia / f7d56ec
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 400 / Auto-skipped: 0


pgsql_mnesia_23 / pgsql_mnesia / f7d56ec
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


pgsql_mnesia_24 / pgsql_mnesia / f7d56ec
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


mysql_redis_24 / mysql_redis / f7d56ec
Reports root/ big
OK: 3219 / Failed: 0 / User-skipped: 147 / Auto-skipped: 0


elasticsearch_and_cassandra_24 / elasticsearch_and_cassandra_mnesia / f7d56ec
Reports root/ big
OK: 1854 / Failed: 0 / User-skipped: 367 / Auto-skipped: 0


mssql_mnesia_24 / odbc_mssql_mnesia / f7d56ec
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


riak_mnesia_24 / riak_mnesia / f7d56ec
Reports root/ big
OK: 1697 / Failed: 0 / User-skipped: 366 / Auto-skipped: 0

@mongoose-im
Copy link
Collaborator

mongoose-im commented Mar 30, 2022

small_tests_24 / small_tests / 6315f56
Reports root / small


small_tests_23 / small_tests / 6315f56
Reports root / small


dynamic_domains_mysql_redis_24 / mysql_redis / 6315f56
Reports root/ big
OK: 2833 / Failed: 0 / User-skipped: 150 / Auto-skipped: 0


dynamic_domains_pgsql_mnesia_24 / pgsql_mnesia / 6315f56
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


dynamic_domains_pgsql_mnesia_23 / pgsql_mnesia / 6315f56
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


ldap_mnesia_23 / ldap_mnesia / 6315f56
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 400 / Auto-skipped: 0


dynamic_domains_mssql_mnesia_24 / odbc_mssql_mnesia / 6315f56
Reports root/ big
OK: 2850 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


ldap_mnesia_24 / ldap_mnesia / 6315f56
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 400 / Auto-skipped: 0


internal_mnesia_24 / internal_mnesia / 6315f56
Reports root/ big
OK: 1547 / Failed: 0 / User-skipped: 359 / Auto-skipped: 0


pgsql_mnesia_24 / pgsql_mnesia / 6315f56
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


elasticsearch_and_cassandra_24 / elasticsearch_and_cassandra_mnesia / 6315f56
Reports root/ big
OK: 1854 / Failed: 0 / User-skipped: 367 / Auto-skipped: 0


pgsql_mnesia_23 / pgsql_mnesia / 6315f56
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


mysql_redis_24 / mysql_redis / 6315f56
Reports root/ big
OK: 3219 / Failed: 0 / User-skipped: 147 / Auto-skipped: 0


mssql_mnesia_24 / odbc_mssql_mnesia / 6315f56
Reports root/ big
OK: 3224 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


riak_mnesia_24 / riak_mnesia / 6315f56
Reports root/ big
OK: 1697 / Failed: 0 / User-skipped: 366 / Auto-skipped: 0

@chrzaszcz chrzaszcz marked this pull request as ready for review March 30, 2022 12:03
Copy link
Collaborator

@NelsonVides NelsonVides left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, kill dead code with 🔥
(Though now I'm left thinking, what was the purpose of such reverse proxy in MIM, how was it working and why was it added... 🤔 )

@chrzaszcz
Copy link
Member Author

chrzaszcz commented Mar 30, 2022

Great, kill dead code with 🔥 (Though now I'm left thinking, what was the purpose of such reverse proxy in MIM, how was it working and why was it added... 🤔 )

The purpose was to use it as a regular web server (or to attach your custom Cowboy handlers AFAIR). We dropped it when introducing the TOML config because it was super-complicated... and we already removed the listener part of the code.

@gustawlippa
Copy link
Contributor

There's one place with something relating to this I think:

mod_pubsub_db_backend, mod_revproxy_dynamic,

This module is dead - it was impossible to configure it since version 4.2.0

Also: add missing cleanup revealed by the removed test suite.
@chrzaszcz
Copy link
Member Author

chrzaszcz commented Mar 30, 2022

There's one place with something relating to this I think:

mod_pubsub_db_backend, mod_revproxy_dynamic,

Done

@mongoose-im
Copy link
Collaborator

mongoose-im commented Mar 30, 2022

small_tests_23 / small_tests / ab4b9fc
Reports root / small


small_tests_24 / small_tests / ab4b9fc
Reports root / small


dynamic_domains_pgsql_mnesia_24 / pgsql_mnesia / ab4b9fc
Reports root/ big
OK: 2851 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


dynamic_domains_mysql_redis_24 / mysql_redis / ab4b9fc
Reports root/ big
OK: 2834 / Failed: 0 / User-skipped: 150 / Auto-skipped: 0


dynamic_domains_pgsql_mnesia_23 / pgsql_mnesia / ab4b9fc
Reports root/ big
OK: 2851 / Failed: 0 / User-skipped: 133 / Auto-skipped: 0


dynamic_domains_mssql_mnesia_24 / odbc_mssql_mnesia / ab4b9fc
Reports root/ big
OK: 2856 / Failed: 1 / User-skipped: 133 / Auto-skipped: 0

muc_SUITE:register:user_unregisters_nick
{error,
  {{assertion_failed,assert,is_iq_result,
     [{xmlel,<<"iq">>,
        [{<<"type">>,<<"set">>},
         {<<"id">>,<<"62f5c96576f5ddc4f0d0a86352c751c9">>},
         {<<"to">>,<<"groupchats.domain.example.com">>}],
        [{xmlel,<<"query">>,
           [{<<"xmlns">>,<<"jabber:iq:register">>}],
           [{xmlel,<<"x">>,
            [{<<"xmlns">>,<<"jabber:x:data">>},
             {<<"type">>,<<"submit">>}],
            [{xmlel,<<"field">>,
               [{<<"type">>,<<"hidden">>},
                {<<"var">>,<<"FORM_TYPE">>}],
               [{xmlel,<<"value">>,[],
                  [{xmlcdata,<<"jabber:iq:register">>}]}]},
             {xmlel,<<"field">>,
               [{<<"type">>,<<"text-single">>},
                {<<"var">>,<<"nick">>}],
               [{xmlel,<<"value">>,[],
                  [{xmlcdata,
                     <<"thirdwitch1room-59efac9b8a">>}]}]}]}]}]}],
     {xmlel,<<"iq">>,
       [{<<"from">>,<<"groupchats.domain.example.com">>},
        {<<"to">>,
         <<"[email protected]/res1">>},
        {<<"type">>,<<"error">>},
        {<<"xml:lang">>,<<"en">>},
        {<<"id">>,<<"62f5c96576f5ddc4f0d0a86352c751c9">>}],
       [{xmlel,<<"query">>,
          [{<<"xmlns">>,<<"jabber:iq:register">>}],
          [{xmlel,<<"x">>,
             [{<<"xmlns">>,<<"jabber:x:data">>},
            {<<"type">>,<<"submit">>}],
             [{xmlel,<<"field">>,
              [{<<"type">>,<<"hidden">>},
               {<<"var">...

Report log


ldap_mnesia_24 / ldap_mnesia / ab4b9fc
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 401 / Auto-skipped: 0


internal_mnesia_24 / internal_mnesia / ab4b9fc
Reports root/ big
OK: 1547 / Failed: 0 / User-skipped: 360 / Auto-skipped: 0


ldap_mnesia_23 / ldap_mnesia / ab4b9fc
Reports root/ big
OK: 1506 / Failed: 0 / User-skipped: 401 / Auto-skipped: 0


pgsql_mnesia_23 / pgsql_mnesia / ab4b9fc
Reports root/ big
OK: 3225 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


elasticsearch_and_cassandra_24 / elasticsearch_and_cassandra_mnesia / ab4b9fc
Reports root/ big
OK: 1854 / Failed: 0 / User-skipped: 368 / Auto-skipped: 0


mysql_redis_24 / mysql_redis / ab4b9fc
Reports root/ big
OK: 3220 / Failed: 0 / User-skipped: 147 / Auto-skipped: 0


pgsql_mnesia_24 / pgsql_mnesia / ab4b9fc
Reports root/ big
OK: 3225 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


mssql_mnesia_24 / odbc_mssql_mnesia / ab4b9fc
Reports root/ big
OK: 3225 / Failed: 0 / User-skipped: 142 / Auto-skipped: 0


riak_mnesia_24 / riak_mnesia / ab4b9fc
Reports root/ big
OK: 1697 / Failed: 0 / User-skipped: 367 / Auto-skipped: 0

@gustawlippa gustawlippa merged commit 7cbb30d into master Mar 30, 2022
@gustawlippa gustawlippa deleted the delete-mod-revproxy branch March 30, 2022 15:17
@Premwoik Premwoik added this to the 5.1.0 milestone May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants